Hey @hjiangsu I was just going to comment on this PR that I had run across another Flutter app which uses video_player directly with their own controls on top of it. I was going to link their code in case it had anything helpful.

https://github.com/j-fbriere/squawker/blob/master/lib/tweet/_video_controls.dart

Then I noticed that you had already marked this PR ready for review? Is that still the case? Would it be worth taking it for a test?

Ahh yeah, this PR is ready for review but I was going to wait until 0.6.0 to merge it in! I think we can take a look at how their implementation looks like, and take ideas to improve on the current implementation I have 😅

Just to double-check, your video player is not being used for YouTube videos, right? We would have to introduce a new step which retrieves the raw video URL to pass to the player?

Just to double-check, your video player is not being used for YouTube videos, right? We would have to introduce a new step which retrieves the raw video URL to pass to the player?

Correct! This is only currently being used for the general video player but I would like to extend this to YouTube videos as well.

One idea we can do here is to use a package like https://pub.dev/packages/youtube_explode_dart to possibly retrieve metadata for a given YouTube url (without the need to provide an API key). However, my concern with using something like youtube_explode_dart is that it can be subject to break if YouTube itself changes their internal logic.

A (potentially) more robust but less trivial solution would be to convert the youtube link to a piped link (or similar) and retrieve video metadata there. If there's no dart library for this, we might have to build one ourselves to call the proper API endpoints.

Sounds good! You should be good to merge this after the next general release and we can start getting our hands on it.

Just merged this in - let me know if you encounter any quirks when using this!